Skip to content
This repository has been archived by the owner on May 16, 2023. It is now read-only.

[logstash] update logstash default values for memory requirements #367

Merged
merged 2 commits into from
Nov 13, 2019
Merged

[logstash] update logstash default values for memory requirements #367

merged 2 commits into from
Nov 13, 2019

Conversation

jmlrt
Copy link
Member

@jmlrt jmlrt commented Nov 13, 2019

  • Chart version not bumped (the versions are all bumped and released at the same time)
  • README.md updated with any new values or changes
  • Updated template tests in ${CHART}/tests/*.py
  • Updated integration tests in ${CHART}/examples/*/test/goss.yaml

@jmlrt jmlrt added the enhancement New feature or request label Nov 13, 2019
@jmlrt jmlrt self-assigned this Nov 13, 2019
@jmlrt
Copy link
Member Author

jmlrt commented Nov 13, 2019

jenkins test this please

@jmlrt jmlrt requested a review from fatmcgav November 13, 2019 08:42
fatmcgav
fatmcgav approved these changes Nov 13, 2019
View reviewed changes
Copy link
Contributor

@fatmcgav fatmcgav left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jmlrt jmlrt merged commit 7b47ceb into elastic:master Nov 13, 2019
@jmlrt jmlrt deleted the logstash-memory branch November 13, 2019 09:03
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Reviewers

@fatmcgav fatmcgav fatmcgav approved these changes

Assignees

@jmlrt jmlrt

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jmlrt @fatmcgav @adrianlzt